Skip to content

feat: add VS Code/Copilot integration for tool-agnostic sessions (fixes #7)#16

Merged
josealekhine merged 7 commits intoActiveMemory:mainfrom
bilersan:pr/issue-7-copilot
Feb 19, 2026
Merged

feat: add VS Code/Copilot integration for tool-agnostic sessions (fixes #7)#16
josealekhine merged 7 commits intoActiveMemory:mainfrom
bilersan:pr/issue-7-copilot

Conversation

@bilersan
Copy link
Contributor

@bilersan bilersan commented Feb 17, 2026

Summary

Adds first-class VS Code and GitHub Copilot support to ctx, delivering the tool-agnostic session persistence pipeline proposed in #7.

This PR introduces three tightly coupled components that together enable Copilot Chat to read and write ctx sessions without requiring users to leave their editor.

What's Included

1. Tool-Agnostic Session Parser (internal/recall/parser/markdown.go)

  • MarkdownSessionParser that reads .context/sessions/*.md files
  • Handles multiple header formats (# Session, ## , YAML frontmatter)
  • Extracts human/assistant turns, tool calls, code blocks, and metadata
  • Comprehensive test suite (390 lines)

2. Copilot Hook + Init Updates (internal/cli/hook/, internal/cli/initialize/)

  • ctx hook copilot --write generates .github/copilot-instructions.md with ctx-aware system prompt
  • ctx init now creates .context/sessions/ directory for session storage
  • toolConfigFiles map for extensible tool config discovery

3. VS Code Chat Participant Extension (editors/vscode/)

  • @ctx chat participant with 10 slash commands: /init, /load, /save, /status, /list, /search, /config, /help, /version, /recall
  • CancellationToken support for all operations
  • Captures both stdout and stderr from ctx CLI
  • 11 unit tests (vitest), esbuild bundler, 7.6 KB output
  • Requires VS Code 1.93+

What's Intentionally Excluded

This PR is scoped to VS Code and GitHub Copilot only. The following exist on the feature branch but are held back for separate PRs:

  • Cursor/Windsurf tool support and config generation
  • --tool flag on ctx init
  • Compliance test suite and formatting fixes
  • PowerShell/OneDrive boundary scripts

Testing

  • Go: go build ./... clean, all new parser and hook tests pass
  • Extension: npm run build clean (7.6 KB), 11/11 vitest tests pass, npm audit 0 vulnerabilities
  • DCO: All 4 commits signed off

Connects To

Issue #7 — specifically the proposed architecture of ctx hook <tool> --write generating tool-specific instruction files, with editor extensions acting as the bridge between the tool's chat interface and ctx's session storage.

@josealekhine
Copy link
Member

josealekhine commented Feb 18, 2026

This is very much appreciated 🙏

Will check this out as soon as I can.

Thanks for your contribution.

@josealekhine josealekhine added the enhancement New feature or request label Feb 18, 2026
@josealekhine
Copy link
Member

josealekhine commented Feb 18, 2026

@bilersan there are minor linter issues; not a blocker, but would be cool if you can fix them before I jump in 🙏 .

@bilersan
feat(recall): add MarkdownSessionParser for tool-agnostic sessions
adfd06a 
Add tool-agnostic session parser that reads structured Markdown files
from .context/sessions/, enabling non-Claude tools (Copilot, Cursor,
Windsurf) to participate in the recall/journal pipeline.

Changes:
- Add config constants: ToolClaudeCode, ToolMarkdown, DirSessions
- Implement MarkdownSessionParser with support for multiple header
  formats (# Session: YYYY-MM-DD - Topic, etc.)
- Register markdown parser alongside ClaudeCodeParser
- Scan .context/sessions/ for markdown session files
- Replace hardcoded 'claude-code' strings with config constants
- Add comprehensive parser tests (390 lines)

Refs: ActiveMemory#7
Signed-off-by: ersan bilik <ersanbilik@gmail.com>
@bilersan
feat(hook): add copilot --write flag and init sessions dir
36b83de 
Add --write flag to 'ctx hook copilot' that generates
.github/copilot-instructions.md directly instead of just printing
the configuration snippet. The generated instructions teach Copilot
Chat (agent mode) to:
- Read .context/ files in priority order at session start
- Persist session summaries to .context/sessions/
- Proactively update context files during work
- Respond to memory queries with structured readbacks

Also creates .context/sessions/ during ctx init so the session
persistence workflow is ready out of the box for all tools.

Refs: ActiveMemory#7
Signed-off-by: ersan bilik <ersanbilik@gmail.com>
@bilersan
feat(editors): add VS Code Chat Participant extension
36d3b78 
Add @ctx chat participant for VS Code Copilot Chat that wraps ctx
CLI commands as slash commands. Users can type @ctx /init, @ctx /status,
@ctx /drift, etc. directly in the Chat panel.

Slash commands:
- /init    - Initialize .context/ + auto-generate copilot-instructions.md
- /status  - Show context summary with token estimate
- /agent   - Print AI-ready context packet
- /drift   - Detect stale or invalid context
- /recall  - Browse and search session history
- /hook    - Generate AI tool integration configs
- /add     - Add task, decision, or learning
- /load    - Output assembled context Markdown
- /compact - Archive completed tasks
- /sync    - Reconcile context with codebase

The extension executes the ctx binary and streams results back to
chat. Includes CancellationToken support for aborting long-running
calls, follow-up suggestions, and natural language routing.

Also includes:
- 11 colocated unit tests (vitest)
- esbuild ^0.25.0 (resolves GHSA-67mh-4wv8-2f99)

Refs: ActiveMemory#7
Signed-off-by: ersan bilik <ersanbilik@gmail.com>
@bilersan
docs: update integration and getting-started docs for Copilot
396c9ab 
- Add VS Code Chat Participant section to integrations.md with
  build/install instructions and slash command reference
- Document copilot --write flag in cli-reference.md
- Add Copilot session persistence to first-session.md
- Update getting-started.md with VS Code extension install steps
- Fix vsix version reference (0.6.0 -> 0.7.0)

Refs: ActiveMemory#7
Signed-off-by: ersan bilik <ersanbilik@gmail.com>
@bilersan
chore(vsix): update icon, marketplace metadata, and vscodeignore
19b2ca1
@bilersan
fix: resolve golangci-lint issues (gosec, goconst, unused)
2a82b73 
- Add filepath.Clean for os.ReadFile in hook/run.go (gosec G304)
- Export ToolConfigFiles var in hook/run.go (unused)
- Add goconst nolint directives for trivial yes comparisons
@bilersan
fix: exclude gosec G304 from test files in golangci-lint config
a2ad811 
Test files construct paths from test constants, not user input.
This matches the goconst exclusion already in place.
@bilersan
Copy link
Contributor Author

bilersan commented Feb 18, 2026

@josealekhine Linter issues are fixed! Here's what was done:

Rebased on latest main (af15989) — clean, no conflicts.

Lint fixes (3 new commits):

  • chore(vsix): Updated VSIX icon (128px favicon), marketplace metadata (homepage, bugs, galleryBanner), and .vscodeignore
  • fix: resolve golangci-lint issues: filepath.Clean for gosec G304, exported unused var, goconst nolint directives for trivial yes comparisons
  • fix: exclude gosec G304 from test files: Added rule in .golangci.yml — test files construct paths from constants, not user input

Verification:

  • golangci-lint run --timeout=5m → 0 issues
  • go build ./... → clean
  • All pre-existing test failures are upstream (confirmed by running same tests against main)

Ready for review whenever you have time 🙏

josealekhine
josealekhine approved these changes Feb 19, 2026
View reviewed changes
Copy link
Member

@josealekhine josealekhine left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR-16 Review: VS Code / Copilot Integration

Validation Summary

Step Result
PR-16.1 Baseline capture (main) 250 entries, 289 markdown files
PR-16.2 Apply PR + full test suite All tests pass (4.064s parser pkg)
PR-16.3 Semantic diff (journal pipeline) Zero diff — identical output
PR-16.4 Code quality review Approve with minor suggestions

What's Good

  • MarkdownSessionParser cleanly implements SessionParser interface. Registration, matching, parsing all correct.
  • Backward compatibility confirmed: journal pipeline produces byte-identical output on main vs this branch.
  • Copilot instructions template is comprehensive — read order, session persistence format, context update table, self-check prompt.
  • VS Code extension is well-structured: cancellation token handling, configurable path, follow-up suggestions, natural language fallback.
  • Test coverage is solid: 391 lines of Go parser tests (23+ subtests), 185 lines of VS Code tests (7 runCtx scenarios).
  • Constants extraction (ToolClaudeCode, ToolMarkdown, RoleUser, etc.) improves maintainability.
  • Documentation is thorough — integrations.md expanded from 12 to ~100 lines, getting-started and first-session updated.

Minor Suggestions (Non-Blocking)

  1. Map iteration order (markdown.go:185): for heading, body := range sections iterates map[string]string non-deterministically. The bodyParts slice (and resulting assistant message text) may vary in order between runs. Consider sorting section keys or using an ordered representation. Not a correctness bug — content is identical regardless — but affects reproducibility.

  2. --no-color inconsistency in VS Code extension: handleAgent (L142) and handleLoad (L269) don't pass --no-color, while handleStatus, handleDrift, handleCompact, handleSync, and handleInit all do. These commands may not produce colored output today, but adding the flag would be more consistent.

  3. Potential double-counting in query.go: If .context/sessions/ is also passed as an additionalDirs argument, sessions could appear twice. Low probability in practice but could be guarded with a seen-paths set.

  4. handleHook always passes --write (extension.ts:207): Users can't preview hook output via the extension — it always writes to disk. Consider making --write optional or adding a dry-run note.

Verdict

Approve. The PR is well-structured, fully tested, backward-compatible, and follows project conventions. The minor items above are polish — none block merge.

@bilersan
Copy link
Contributor Author

bilersan commented Feb 19, 2026

image

@josealekhine josealekhine merged commit 27be465 into ActiveMemory:main Feb 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@josealekhine josealekhine josealekhine approved these changes

Assignees

@bilersan bilersan

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bilersan @josealekhine

Comments